Skip to content

Commit

Permalink
Make msvc compile by adding back the ifdefs in gelu_backward
Browse files Browse the repository at this point in the history
  • Loading branch information
dagelf committed Apr 22, 2024
1 parent 8ca8ba8 commit b1c08b9
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions train_gpt2.c
Original file line number Diff line number Diff line change
Expand Up @@ -378,8 +378,10 @@ void leaky_relu_forward(float* out, float* inp, int N) {
}

// we want to use -Ofast optimization, but sadly GeLU breaks, so disable this flag just for it (#168)
#pragma float_control(precise, on, push) // On msvc /fp:fast is a lot faster, but the expf inside coshf breaks the model
__attribute__((optimize("no-finite-math-only"))) // same for gcc -Ofast
#pragma float_control(precise, on, push)
#if defined(__GNUC__) && !defined(__clang__)
__attribute__((optimize("no-finite-math-only")))
#endif
void gelu_backward(float* dinp, float* inp, float* dout, int N) {

float min_x = FLT_MAX;
Expand Down

0 comments on commit b1c08b9

Please sign in to comment.