[PATCH] D100179: [GVN][NFC] Refactor code and add description for GVN object

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

The bit that was split off and committed as rGc35e8185d8c170c20e28956e0c9f3c1be895fefb <https://reviews.llvm.org/rGc35e8185d8c170c20e28956e0c9f3c1be895fefb> was not an NFC, it broke compilation of one file for me: https://martin.st/temp/vlc-preproc.c

  $ clang -target armv7-w64-mingw32 -w -c -O2 vlc-preproc.c 
  clang: ../lib/Transforms/Utils/VNCoercion.cpp:475: llvm::Value* llvm::VNCoercion::getLoadValueForLoad(llvm::LoadInst*, unsigned int, llvm::Type*, llvm::Instruction*, const llvm::DataLayout&): Assertion `SrcVal-isSimple() && "Cannot widen volatile/atomic load!"' failed.

I'd suggest reverting that patch until this issue is sorted out.


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

https://reviews.llvm.org/D100179