[PATCH] D17377: [ADT]Fix AddPointer() in FoldingSet

Eugene via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 13:57:45 PST 2016


kevgs added a comment.

Linux AMD64, but I have no problems with that code. I just thought it was incorrect and feel pretty silly now. However, patched method looks nicer and it's probably faster because compiler is not forced to put variable on the stack ;)

Anyway, thanks four your help and sorry for taking your time on such a stupid case.


Repository:
  rL LLVM

http://reviews.llvm.org/D17377





More information about the llvm-commits mailing list