gulfem added inline comments.
================ Comment at: llvm/lib/Transforms/Utils/RelLookupTableGenerator.cpp:76-77 + + /// If lookup table has more than one user, + /// do not generate a relative lookup table. + if (!GlobalVar.hasOneUser()) ---------------- leonardchan wrote: > What's the reason for why the number of users matters? We generate one lookup table per switch statement whenever possible. I think there is only one use of that lookup table which is the`GetElementPtr` instruction. That is why I checked for one use. Do you see any issue with that? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94355/new/ https://reviews.llvm.org/D94355 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
