[PATCH] D66014: [analyzer] Avoid unnecessary enum range check on LValueToRValue casts

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 23 06:47:30 PDT 2019


Szelethus added a comment.

What I meant is that the diff has to be made against the master branch or I won't be able to apply it locally.  Say your repo is structured like this:

  * dcba2 (HEAD -> my_fix)
  * dcba1
  * abcd4 (master)
  * abcd3
  * abcd2
  * abcd1

Then the diff has to be made with `git diff master -U99999 > my_fix.diff` and uploaded here, or with the use of arcanist (which I still find incredibly inconvenient). If you only upload the latest commit, I'd be missing `dcba1`.

If you have a lot of changes you want to submit, like this:

  * hjkd2 (enable_feature_by_default)
  * hjkd1
  * haha1 (my_feature)
  * dcba2 (HEAD -> my_fix)
  * dcba1
  * abcd4 (master)

You should create a revision for each branch, diff them  against each other (`git checkout my_feature && git diff my_fix -U99999 > my_feature.diff`), and mark them as dependencies of each other on the right hand panel.

It is kinda inconvenient, I'll admit :^)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66014/new/

https://reviews.llvm.org/D66014





More information about the cfe-commits mailing list